Skip to content

Correcting minimum bid amount to 10_000 CSPR from 100_000 CSPR#5158

Merged
sacherjj merged 1 commit intocasper-network:devfrom
sacherjj:updating_min_bid_back_to_10k
Mar 13, 2025
Merged

Correcting minimum bid amount to 10_000 CSPR from 100_000 CSPR#5158
sacherjj merged 1 commit intocasper-network:devfrom
sacherjj:updating_min_bid_back_to_10k

Conversation

@sacherjj
Copy link
Copy Markdown
Contributor

Please consider the following when creating a PR:

  • Provide a useful description of this PR, suitably verbose, aimed at helping reviewers and contributors
  • Update all relevant changelogs
  • Provide a link to the GitHub issue relating to this PR
  • Identify if any downstream impact as in to, SDKs, SmartContracts etc

Copy link
Copy Markdown
Contributor

@mrkara mrkara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you!

Also, we should inform/warn the validators about this new chainspec value and its behavior because it can practically kill a validator if they don't have this minimum amount at the point of upgrade or if they go below this amount at some point after the upgrade.

# Minimum bid amount allowed in motes. Withdrawing one's bid to an amount strictly less than
# the value specified will be treated as a full unbond of a validator and their associated delegators
minimum_bid_amount = 100_000_000_000_000
minimum_bid_amount = 10_000_000_000_000
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need local chainspec to stay in sync with this value

@devendran-m devendran-m linked an issue Mar 13, 2025 that may be closed by this pull request
@sacherjj sacherjj merged commit 7172e75 into casper-network:dev Mar 13, 2025
2 checks passed
@sacherjj sacherjj deleted the updating_min_bid_back_to_10k branch March 13, 2025 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Correct minimum bid amount to 10_000 CSPR from 100_000 CSPR

4 participants